Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 183 #210

Merged
merged 2 commits into from
Mar 5, 2015
Merged

Issue 183 #210

merged 2 commits into from
Mar 5, 2015

Conversation

AgelxNash
Copy link
Contributor

#183

В общем теперь можно вызвать метод getDocumentObject() с параметром $method отличным от id и alias, чтобы загрузить данные из какой-то своей таблицы при помощи плагина на событии OnBeforeLoadDocumentObject. Даже если оставить штатные значения параметров метода, то при помощи этого события можно упростить обязательные SQL запросы выкинув из них проверку принадлежности документа к группе. Или выкинуть редирект на 404 страницу, если вдруг запрошенного документа через метод getDocumentObject не оказалось (например, в исходнике запрятался id).

Аналогичным образом, можно дозагрузить данные в documentObject через событие OnAfterLoadDocumentObject. Это событие тоже добавлено специально, т.к. вдруг какая-то связь хранится в ТВ, которая может понадобиться в плагине.

Ну и собственно из-за чего все это затеивалось - событие OnLoadDocumentObject, как и 2 новых события (OnBeforeLoadDocumentObject, OnAfterLoadDocumentObject) теперь принимает новые аргументы method, identifier и текущее значение documentObject

Dmi3yy added a commit that referenced this pull request Mar 5, 2015
@Dmi3yy Dmi3yy merged commit 215285b into Dmi3yy:master Mar 5, 2015
@AgelxNash AgelxNash deleted the issue-183 branch March 5, 2015 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants